Skip to content

fix(agent): Flush proxy ingest uploads live#3042

Merged
charlesvien merged 1 commit into
mainfrom
codex/agent-proxy-writepath-fix
Jul 1, 2026
Merged

fix(agent): Flush proxy ingest uploads live#3042
charlesvien merged 1 commit into
mainfrom
codex/agent-proxy-writepath-fix

Conversation

@charlesvien

Copy link
Copy Markdown
Member

Agent-proxy write-leg uploads could stay open for five minutes, delaying Redis/SSE events until the request closed.

  1. Use 1s stream windows for agent-proxy ingest
  2. Preserve the 5m streaming window for Django ingest
  3. Add fake-timer coverage for proxy upload rotation

Verification:

  • pnpm --filter @posthog/agent test src/server/event-stream-sender.test.ts
  • pnpm --filter @posthog/agent typecheck

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 083f5f7.

@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "flush proxy ingest uploads sooner" | Re-trigger Greptile

@charlesvien charlesvien added the Stamphog This will request an autostamp by stamphog on small changes label Jul 1, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small, targeted fix that shortens the stream window for proxy ingest from 5 minutes to 1 second using the already-existing usingProxy flag. Accompanied by a focused test that validates the behavior. No showstoppers.

@charlesvien charlesvien merged commit b38b7f6 into main Jul 1, 2026
23 checks passed
@charlesvien charlesvien deleted the codex/agent-proxy-writepath-fix branch July 1, 2026 02:04
charlesvien added a commit that referenced this pull request Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant